Skip to content

Conversation

martakam
Copy link
Member

@martakam martakam commented Nov 8, 2026

This PR adds linting to the backend using ESLint, in a very similar fashion as the frontend.
The most scrict rulesets have been applied, because we strive to write code of high quality (hopefully).

typescript-eslint does not support TypeScript v7 yet, so the backend has been downgraded to v6.

All issues highlighted by ESLint have been fixed.

Closes #109

@martakam martakam added the enhancement New feature or request label Nov 8, 2026
@martakam martakam added this to the Deliverable 4 milestone Nov 8, 2026
@martakam martakam self-assigned this Nov 8, 2026
@martakam martakam requested a review from a team as a code owner November 8, 2026 17:36
@johahor
Copy link
Member

johahor commented Nov 10, 2026

Looks good! Nice to have the same strict linting in the backend.

This branch is now in conflict with main, I'll fix them, and make a helper function for the work you had to do in this commit to satisfy ESlint: 2ff0b8e

@johahor
Copy link
Member

johahor commented Nov 10, 2026

4 tests are failing. 3 that are being fixed in #107 , and one that I'll create an issue for. It's in the FavoriteButton.test.tsx test. Sometimes it works, and sometimes it fails.. weird. Approving this PR

@johahor johahor merged commit 48944c4 into main Nov 10, 2026
@martakam martakam deleted the feat/backend-eslint branch November 12, 2026 21:31
@martakam martakam restored the feat/backend-eslint branch November 12, 2026 21:31
@martakam martakam deleted the feat/backend-eslint branch November 12, 2026 21:31
Sign in to join this conversation on GitHub.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: add linting to backend
2 participants